Skip to content

refactor(runtime): migrate alert and notification state (#467)#468

Merged
Orinks merged 2 commits intodevfrom
eli/issue-467-runtime-state-phase2
Mar 16, 2026
Merged

refactor(runtime): migrate alert and notification state (#467)#468
Orinks merged 2 commits intodevfrom
eli/issue-467-runtime-state-phase2

Conversation

@Orinks
Copy link
Owner

@Orinks Orinks commented Mar 16, 2026

Summary

  • migrate alert runtime state into the unified runtime store with per-section legacy fallback
  • migrate notification event state into the unified runtime store while preserving AFD and severe-risk continuity
  • keep portable/settings copy flows excluding runtime state and add migration-focused regression coverage

Testing

  • pytest tests/test_runtime_state_manager.py tests/test_alert_manager.py tests/test_notification_event_manager.py tests/test_split_notification_timers.py tests/test_settings_dialog_portable_copy.py -q
  • ruff check src/accessiweather/runtime_state.py src/accessiweather/alert_manager.py src/accessiweather/notifications/notification_event_manager.py src/accessiweather/ui/main_window_notification_events.py src/accessiweather/app_initialization.py tests/test_runtime_state_manager.py tests/test_alert_manager.py tests/test_notification_event_manager.py tests/test_split_notification_timers.py

Closes #467

@Orinks Orinks merged commit 3530cd8 into dev Mar 16, 2026
3 checks passed
@Orinks Orinks deleted the eli/issue-467-runtime-state-phase2 branch March 16, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant